Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Companion: MessageId (ump, dmp) #2504

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Companion: MessageId (ump, dmp) #2504

wants to merge 2 commits into from

Conversation

bkontur
Copy link
Contributor

@bkontur bkontur commented May 2, 2023

Polkadot companion: paritytech/polkadot#7161

@bkontur bkontur added B0-silent Changes should not be mentioned in any release notes A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels May 2, 2023
@@ -24,7 +24,9 @@
pub mod migration;

use codec::{Decode, DecodeLimit, Encode};
use cumulus_primitives_core::{relay_chain::BlockNumber as RelayBlockNumber, DmpMessageHandler};
use cumulus_primitives_core::{
Copy link
Member

@ggwpez ggwpez May 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dmp-queue pallet is being deleted here #2157
Not sure if that is relevant to you, just pointing it out.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, thank you, then better :),
I will check

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/cumulus/-/jobs/2838158

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants